-
Notifications
You must be signed in to change notification settings - Fork 8.4k
soc: nxp: imxrt118x: update MPU configuration #80722
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
soc: nxp: imxrt118x: update MPU configuration #80722
Conversation
|
Hello @nxp-shelley, and thank you very much for your first pull request to the Zephyr project! |
|
@hakehuang , can you run our tests on this board, and ensure they all pass with this PR? @lucien-nxp , can you please review this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @nxp-shelley for resolving this.
But can you review these comments about the FlexSPI region sizes?
I did replicate the issue, and see this PR resolves it. When building the ADC test below from the main branch, nothing prints out the console. With this patch, the test passes and can be reflashed.
west build -b mimxrt1180_evk//cm33 tests/drivers/adc/adc_api/ --pristine
@DerekSnell it works. |
fixes: zephyrproject-rtos#80721 Updated mpu region address to secure address. Signed-off-by: Xiaoli Ji <[email protected]>
a1cbe10 to
14b1e50
Compare
DerekSnell
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, @nxp-shelley . Thank you for this fix!
|
Hi @nxp-shelley! To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge. Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁 |
fixes: #80721
Updated mpu region address to secure address.